Skip to content

[SYCL][NativeCPU] Process nativecpu_utils with prepare_builtins #17850

New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Merged
merged 1 commit into from
Apr 14, 2025

Conversation

hvdijk
Copy link
Contributor

@hvdijk hvdijk commented Apr 4, 2025

This ensures that functions have the right linkage.

Several functions are marked as used to prevent them from being removed as dead code before the work item loop pass and PrepareSYCLNativeCPUPass run.

@hvdijk hvdijk requested review from a team as code owners April 4, 2025 11:00
@hvdijk hvdijk requested a review from uditagarwal97 April 4, 2025 11:00
@hvdijk hvdijk force-pushed the nativecpu-prepare-builtins branch from 2af6bd1 to 45038dc Compare April 4, 2025 11:05
@hvdijk hvdijk temporarily deployed to WindowsCILock April 4, 2025 11:05 — with GitHub Actions Inactive
This ensures that functions have the right linkage.

Several functions are marked as used to prevent them from being removed
as dead code before the work item loop pass runs.
@hvdijk hvdijk force-pushed the nativecpu-prepare-builtins branch from 45038dc to 3ce9b0c Compare April 4, 2025 11:32
@hvdijk hvdijk temporarily deployed to WindowsCILock April 4, 2025 11:33 — with GitHub Actions Inactive
@hvdijk hvdijk temporarily deployed to WindowsCILock April 4, 2025 11:55 — with GitHub Actions Inactive
@hvdijk hvdijk temporarily deployed to WindowsCILock April 4, 2025 11:55 — with GitHub Actions Inactive
@hvdijk
Copy link
Contributor Author

hvdijk commented Apr 14, 2025

ping @intel/llvm-reviewers-runtime

@hvdijk
Copy link
Contributor Author

hvdijk commented Apr 14, 2025

@intel/llvm-gatekeepers This is ready to be merged, thanks.

@steffenlarsen steffenlarsen merged commit 9433a80 into intel:sycl Apr 14, 2025
36 of 37 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants